Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: move to gh actions #85

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

frezbo
Copy link
Member

@frezbo frezbo commented Sep 14, 2023

Move to GitHub actions.

@frezbo frezbo force-pushed the chore/move-to-gh-actions branch 5 times, most recently from c00ce80 to 2ac79c3 Compare September 19, 2023 09:43
@frezbo frezbo force-pushed the chore/move-to-gh-actions branch 3 times, most recently from d1aa699 to 954cd2a Compare September 26, 2023 16:00
@frezbo frezbo force-pushed the chore/move-to-gh-actions branch 7 times, most recently from ed96922 to 4b59b4b Compare September 26, 2023 17:38
branches:
- main
- release-*
schedule:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does the schedule mean?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean not sure which step runs there

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

schedule is cron, runs weekly the reproducibilty, there are if condition to match things

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that reproducibility depends on label, not on schedule event?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ahh missed it

env:
CI_ARGS: --cache-from=type=registry,ref=registry.dev.siderolabs.io/${GITHUB_REPOSITORY}:buildcache --cache-to=type=registry,ref=registry.dev.siderolabs.io/${GITHUB_REPOSITORY}:buildcache,mode=max
pull_request:
types: [opened, synchronize, reopened, labeled]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does it mean it will run the build each time labeled is touched?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only if label is removed and added

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, but technically on any label change? it will re-run all steps?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeh, kind of, but the step will be skipped if label is not matched

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but the build will still run though... I wonder if it should be a separate workflow?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeh, even if separate workflow, it starts a job anyways, so not much different. Well i could have a different one that just triggers on label and skip at job level. I guess it's probably easier to read

if: github.event_name != 'schedule'
run: |
make
- name: reproducibility
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does it mean it runs after build?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, only on schedule or when there's the label

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, but should it be a separate workflow? would it be cleaner?

this will have its own status, as reproducibility is independent of all other steps

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeh,i initially had that, felt like a lot of duplication, so moved it along the same job

Makefile Outdated Show resolved Hide resolved
@frezbo frezbo force-pushed the chore/move-to-gh-actions branch 9 times, most recently from 6de4ea8 to d54ac0d Compare September 26, 2023 19:01
Move to GitHub actions.

Signed-off-by: Noel Georgi <[email protected]>
@frezbo frezbo force-pushed the chore/move-to-gh-actions branch from d54ac0d to 639ddad Compare September 26, 2023 19:06
@frezbo
Copy link
Member Author

frezbo commented Sep 26, 2023

/m

@talos-bot talos-bot merged commit 639ddad into siderolabs:main Sep 26, 2023
3 checks passed
@frezbo frezbo deleted the chore/move-to-gh-actions branch September 26, 2023 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants